Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi platform single repo #6605

Closed

Conversation

jfmherokiller
Copy link

@jfmherokiller jfmherokiller commented Dec 24, 2024

This change will allow users to use the same repo between multiple operating systems when using a file system like btrfs on a portable device like a flashdrive.

I currently have this under draft because i seem to be encountering en error where server.py cant find modules and I dont know why.

Edit: it seems to function if I call server.py directly from inside a window with the env set.

Edit: a full system reboot and btrfs rebalance seems to have made it function as desired i am now opening this pullreq

Checklist:

@jfmherokiller jfmherokiller marked this pull request as ready for review December 24, 2024 08:39
@jfmherokiller
Copy link
Author

jfmherokiller commented Dec 24, 2024

I am currently using it on my daulboot system to allow me to only need to store a single copy of my models and the repo while also making use of the dedup feature of btrfs to keep only single copies of the python bits.

@jfmherokiller
Copy link
Author

forgot to check the styleguide correctly but now all is fixed

@oobabooga
Copy link
Owner

This change is very well implemented, however, it would break existing installs that depend on installer_files/conda and installer_files/env.

This could of course be managed through conditional statements in all modified files, but I don't think enough people would benefit from this to justify the added complexity.

@oobabooga oobabooga closed this Jan 8, 2025
@jfmherokiller
Copy link
Author

jfmherokiller commented Jan 8, 2025

the biggest issue I have observed with my changes by using this to run from a partition that is shared between windows and linux is that mount points need to be the same between linux versions and windows versions.

if you remount the parition/disk containing textgen to a new drive letter or mount point you will be forced to reinstall conda.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants